Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add workflow to run conda-store user journey tests #2895

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Add workflow to run conda-store user journey tests #2895

wants to merge 2 commits into from
+40 −1

Conversation

soapy1
Copy link
Contributor

@soapy1 soapy1 commented Jan 6, 2025

Reference Issues or PRs

fixes #2760

What does this implement/fix?

Put a x in the boxes that apply

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds a feature)
  • Breaking change (fix or feature that would cause existing features not to work as expected)
  • Documentation Update
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • Other (please describe): CI change

@soapy1 soapy1 force-pushed the conda-store-black-box-testing branch 11 times, most recently from 8bb4c2a to 814d7e8 Compare January 7, 2025 22:09
@soapy1
Copy link
Contributor Author

soapy1 commented Jan 8, 2025

depends on conda-incubator/conda-store#1040

@soapy1 soapy1 force-pushed the conda-store-black-box-testing branch from 814d7e8 to b58c5c7 Compare January 8, 2025 19:11
@peytondmurray peytondmurray mentioned this pull request Jan 9, 2025
Merged
3 tasks
@viniciusdc
Copy link
Contributor

Since these tests depend on a live cluster, I suggest extending the already existing local integration tests with this extra check https://github.com/nebari-dev/nebari/blob/main/.github/workflows/test_local_integration.yaml -- since it uses CiRun under the hood you should not be affected by any flakiness of the GH runner in case of constrained resources.

@soapy1 soapy1 force-pushed the conda-store-black-box-testing branch 5 times, most recently from 1cf59dd to 02a86e3 Compare January 14, 2025 17:50
@soapy1 soapy1 force-pushed the conda-store-black-box-testing branch from a204fe3 to 0aad967 Compare January 14, 2025 23:10
@soapy1 soapy1 force-pushed the conda-store-black-box-testing branch from 0aad967 to 56f7d4c Compare January 15, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
🪴 Nebari Project Management
Status: New 🚦
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[ENH] - add conda-store black-box testing to ci
2 participants
@soapy1 @viniciusdc